-
Notifications
You must be signed in to change notification settings - Fork 221
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add libtor exiting logic #6782
Conversation
Added libtor exiting logic to: - log any shutdown errors - remove temporary files - added a failsafe timeout for test-liveness
Test Results (CI) 3 files 129 suites 35m 28s ⏱️ Results for commit b6e029b. ♻️ This comment has been updated with latest results. |
Test Results (Integration tests) 2 files + 2 11 suites +11 53m 53s ⏱️ + 53m 53s For more details on these failures, see this check. Results for commit b6e029b. ± Comparison against base commit f5365ca. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Description
The base node and console wallet have various immediate exit paths through 'process::exit', as well as controlled exit via user/shutdown command.
This implementation
uses a unique but predictable handshake file containing the libtor temporary directory for each base node or console wallet running instance to cover all exit paths. When the main thread exits, via user command or 'process::exit', the libtor temporary files and the handshake file are removed.does away with the temporary libtor data folder and log file and instead uses a fixed or user-controllable location via the--libtor-data-dir
command line option.Added libtor exiting logic to:log any shutdown errors;remove temporary files.Added a failsafe timeout for test-liveness.
Closes #6739 and #6741
Motivation and Context
See #6739 and #6741
How Has This Been Tested?
System-level testing:
What process can a PR reviewer use to test or verify this change?
System-level testing
Code review
Breaking Changes